[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign qu1queee for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Thank you @kevydotvinu for adding your contribution.
A few points related to your PR:

• first it is valid point to ensure that we have once instance of shipwright-build CR
• we need to have an approach where we fix the name of the CR,and validate that name before any reonciliation (shipwright-config)
• we would then include a targetNamespace reconcile function, that deletes the old resources if the target namespace changes
• The change needs a release note and docs update

PR needs rebase.

Hello @jkhelil, thanks for reviewing. Sure, I will add those changes.

@kevydotvinu please let us know if you have the time to work on this PR further. We plan on doing a significant refactor soon which will make this pull request very obsolete and hard to rebase.

IMO, using webhook is a better user experience for this use case.

I will work on it.

@adambkaplan can you remove me from the reviewers for this pull request please?

/close

This PR is pretty old and now obsolete. @k37y if this is something you would like to continue working on, please open a new pull request.

@adambkaplan: Closed this PR.